Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update .github repo to Tier 3 Repository Template #4

Merged
merged 3 commits into from
Aug 2, 2024
Merged

Update .github repo to Tier 3 Repository Template #4

merged 3 commits into from
Aug 2, 2024

Conversation

natalialuzuriaga
Copy link
Contributor

Update .github repo to Tier 3 Repository Template

Problem

We have made significant updates to our repository templates and did not get the chance to update the .github repo to reflect the latest version.

Solution

Updated .github repo to be our Tier 3 Repository Template

@natalialuzuriaga
update to match tier 3
78e25c7 
Signed-off-by: Natalia Luzuriaga <[email protected]>
@natalialuzuriaga
Copy link
Contributor Author

Do we still want example-spdx-headers.md and insert_license_headers.sh?

@natalialuzuriaga
Copy link
Contributor Author

PR for #1 and kinda #2

@decause-gov
Copy link
Contributor

decause-gov commented Jul 25, 2024 •
edited
Loading

Do we still want example-spdx-headers.md and insert_license_headers.sh?

I am +1 to keep these for now, and recommend we explore when/where to include headers in our outbound checklists.

Along with this, (perhaps in another ticket) we should explore adding other headers, such as "We're Hiring" ASCII art as well ;)


                                                    ___       _        
                                                   / (_)     | |        
  __ _  ___   ___ _ __ ___  ___   __ _  _____   __/ / _  ___ | |__  ___ 
 / _` |/ _ \ / __| '_ ` _ \/ __| / _` |/ _ \ \ / / / | |/ _ \| '_ \/ __| 
| (_| | (_) | (__| | | | | \__ \| (_| | (_) \ V / /  | | (_) | |_) \__ \
 \__, |\___(_)___|_| |_| |_|___(_)__, |\___/ \_/_/   | |\___/|_.__/|___/
  __/ |                           __/ |             _/ |        
 |___/                           |___/             |__/

@natalialuzuriaga
Copy link
Contributor Author

License Header work is captured in this ticket: DSACMS/repo-scaffolder#133

Ready for re-review: @decause-gov

@decause-gov decause-gov mentioned this pull request Aug 2, 2024
Closed
decause-gov
decause-gov approved these changes Aug 2, 2024
View reviewed changes
Copy link
Contributor

@decause-gov decause-gov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What an incredible body of work you all have accomplished this summer!!! I haven't seen all of the new workflows in one place like this yet, and it has raised the floor of our baseline significantly! We have added so much information, and automated the gathering of that information as well. Excellent OSPO work, love to see it!

LGTML +1 🚢 ⚓

@decause-gov decause-gov merged commit 0eee03c into main Aug 2, 2024
@decause-gov decause-gov deleted the nat/update branch August 2, 2024 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@decause-gov decause-gov decause-gov approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@natalialuzuriaga @decause-gov